Skip to content

Fix Read the Docs builds #412

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

kurtmckee
Copy link
Contributor

Read the Docs builds began permanently failing on February 26, 2024.

The differential diagnosis between success and failure seems to be that the last successful build used Poetry 1.7.1; the first failing build used Poetry 1.8.1. For reference, Poetry 1.8.1 was released on February 26, 2024.

Read the Docs later updated its "Install dependencies with Poetry" documentation to note that Poetry needs a $VIRTUAL_ENV environment variable set.

This change aligns the Read the Docs config with the documented fix, and simultaneously updates the build OS and Python version, which are both very old.

Read the Docs builds began permanently failing on February 26, 2024.

The differential diagnosis between success and failure
seems to be that the last successful build used Poetry 1.7.1;
the first failing build used Poetry 1.8.1.
For reference, Poetry 1.8.1 was released on February 26, 2024.

Read the Docs later updated its "Install dependencies with Poetry"
documentation to note that Poetry needs a `$VIRTUAL_ENV`
environment variable set.

https://docs.readthedocs.com/platform/stable/build-customization.html#install-dependencies-with-poetry

This change aligns the Read the Docs config with the documented
fix, and simultaneously updates the build OS and Python version,
which are both very old.
Copy link
Collaborator

@p1c2u p1c2u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's very helpful. Thank you for the investigation.

@p1c2u p1c2u merged commit 8506f4e into python-openapi:master Jul 1, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants